Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

compose/dracut: Use a host tmpdir for dracut #1126

Closed
wants to merge 1 commit into from

Conversation

cgwalters
Copy link
Member

@cgwalters cgwalters commented Dec 2, 2017

In unified core mode, this avoids an intense spam of errors from cp
because tmpfs doesn't support the user. xattr namespace, and
since this dracut commit
dracut tries to copy all xattrs, which was just done for IMA.
There's no point to having the SELinux labels or other xattrs
in the initramfs.

The real fix here is dracut should learn to only copy the IMA
xattrs, or even better disable IMA enforcement for the dracut
run or something.

In unified core mode, this avoids an intense spam of errors from `cp`
because `tmpfs` doesn't support the `user.` xattr namespace, and
since [this dracut commit](dracutdevs/dracut@61c761b)
dracut tries to copy all xattrs, which was just done for IMA.
There's no point to having the SELinux labels or other xattrs
in the initramfs.

The real fix here is dracut should learn to *only* copy the IMA
xattrs, or even better disable IMA enforcement for the dracut
run or something.
@cgwalters
Copy link
Member Author

Original discussion: #940 (comment)

@jlebon
Copy link
Member

jlebon commented Dec 4, 2017

Nice! @rh-atomic-bot r+ 11c7607

@rh-atomic-bot
Copy link

⌛ Testing commit 11c7607 with merge f8bc9a1...

@rh-atomic-bot
Copy link

☀️ Test successful - status-atomicjenkins
Approved by: jlebon
Pushing f8bc9a1 to master...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants